Skip to content

feat: experimental logger#16477

Open
ematipico wants to merge 17 commits intomainfrom
feat/logger
Open

feat: experimental logger#16477
ematipico wants to merge 17 commits intomainfrom
feat/logger

Conversation

@ematipico
Copy link
Copy Markdown
Member

@ematipico ematipico commented Apr 24, 2026

Changes

This PR merges all works done with the logger to the main branch

Closes AST-72
Closes AST-43

Testing

Plenty done

Docs

withastro/docs#13787

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 24, 2026

🦋 Changeset detected

Latest commit: 6933454

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions Bot added the semver: minor Change triggers a `minor` release label Apr 24, 2026
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is blocked because it contains a minor changeset. A reviewer will merge this at the next release if approved.

@github-actions github-actions Bot added pkg: integration Related to any renderer integration (scope) pkg: astro Related to the core `astro` package (scope) docs pr labels Apr 24, 2026
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 24, 2026

Merging this PR will not alter performance

✅ 18 untouched benchmarks


Comparing feat/logger (6933454) with main (93101cc)

Open in CodSpeed

@ematipico ematipico added this to the 6.2 milestone Apr 27, 2026
@matthewp
Copy link
Copy Markdown
Contributor

Code looks good to me. Tests failure looks real though, will want for approval before they are all passing.

Comment thread packages/integrations/netlify/test/test-utils.ts Outdated
Comment thread packages/integrations/netlify/test/test-utils.ts Outdated
Co-authored-by: ocavue <ocavue@users.noreply.github.com>
Comment thread packages/astro/test/units/app/logger.test.ts Outdated
Co-authored-by: ocavue <ocavue@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs pr pkg: astro Related to the core `astro` package (scope) pkg: integration Related to any renderer integration (scope) semver: minor Change triggers a `minor` release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants